Skip to content

Add cache size limit support#300

Draft
shaypal5 wants to merge 34 commits into
masterfrom
codex/implement-feature-from-issue-#291
Draft

Add cache size limit support#300
shaypal5 wants to merge 34 commits into
masterfrom
codex/implement-feature-from-issue-#291

Conversation

@shaypal5

Copy link
Copy Markdown
Member

Summary

  • implement cache_size_limit and replacement_policy parameters
  • default to LRU policy and enforce limit in memory core
  • document cache size limiting in README
  • test LRU eviction behaviour
  • update default params test

Testing

  • ruff check .
  • mypy src/cachier
  • pytest -m "memory or pickle" --cov=cachier --cov-report=term
  • ./scripts/test-local.sh all (fails: Docker is not installed)

https://chatgpt.com/codex/tasks/task_e_687a30b9d6dc8323931ef5c545e0c3bb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: config Global/default configuration and settings. area: core-api Decorator API, cache semantics, or public behavior. area: memory In-memory backend. complex issue enhancement help wanted impact: performance Performance, scalability, or benchmark impact. source: codex Opened from a Codex-assisted workflow. status: draft Currently marked as draft. status: open Currently open. type: feature Adds or materially changes user-facing functionality. type: refactor Internal restructuring without intended behavior change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant